Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exporter les membres d'un salon dans un fichier csv #625

Conversation

aulamber
Copy link
Contributor

Cf. #593

@aulamber aulamber force-pushed the add-tab-to-download-csv-file-of-the-list-of-users-in-a-room branch from 21902ba to 18cc90f Compare June 29, 2023 12:41
@aulamber aulamber closed this Jun 30, 2023
@aulamber aulamber reopened this Jun 30, 2023
@odelcroi odelcroi changed the title Add tests to download txt file (list of users in a room) Exporter les membres d'un salon dans un fichier csv Jul 7, 2023
@odelcroi odelcroi closed this Aug 21, 2023
@odelcroi odelcroi reopened this Aug 21, 2023
@estellecomment
Copy link
Contributor

Code written. Tests coming.

Changes :

  • moved the button to the Personnes tab.
  • used AccessibleTooltipButton instead of homemade tooltip code.
  • used FileDownloader instead of homemade download code.
  • renamed things in code
  • shortened user-facing text
  • filename is translated, and contains the name of the room (sanitized)

Francais :
When button is hovered :
image

When button is clicked :
image


Anglais
When button is hovered :
image

When button is clicked :
image

Content of downloaded text file (example) :
@prenom.nom-beta.gouv.fr:agent.dinum.tchap.gouv.fr,@autreprenom.autrenom-machin.gouv.fr:agent.interieur.tchap.gouv.fr

});
});

it("downloads the file when button is clicked", () => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

};
+
+ /* TCHAP */
+ private onExportButtonClick = (ev: ButtonEvent): void => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

along with the button, could it have been a separated component to aliviate the patch footprint?

@odelcroi
Copy link
Member

wonderful

@estellecomment estellecomment merged commit 11961f2 into develop_tchap Aug 29, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants